[Klaud Cold] Update qwen3.5-fp4-b200-sglang-agentic-mtp SGLang image to nightly-dev-cu13-20260907-30705c00 - #2862
Conversation
…to nightly-dev-cu13-20260907-30705c00 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
3d3e633 to
0a43cbe
Compare
|
Thanks for the contribution! Please reach out to respective companies' CODEOWNER to fill in the latest PR_REVIEW_CHECKLIST.md before pinging core maintainer on Slack for review. In order for the signoff PR check bot to trigger, you must follow the PR_REVIEW_CHECKLIST.md template correctly, including the phrase For PR verification, add the PR authors are responsible for ensuring that after merging, all GitHub Action jobs fully pass. A lot of the time, failures are just flakes and simply re-running the failed jobs will fix it. See GitHub's docs on re-running failed jobs 感谢你的贡献！请联系相应公司的 CODEOWNER 填写最新的 PR_REVIEW_CHECKLIST.md，然后再在 Slack 上联系核心维护者进行审阅。为了触发 signoff PR 检查机器人，你必须正确遵循 PR_REVIEW_CHECKLIST.md 模板，包括保留英文语句 如需进行 PR 验证，请为此 PR 添加 PR 作者有责任确保合并后所有 GitHub Action 任务完全通过。 很多时候失败只是偶发抖动（flake），重新运行失败的任务即可解决。参见 GitHub 关于重新运行失败任务的文档 |
1 similar comment
|
Thanks for the contribution! Please reach out to respective companies' CODEOWNER to fill in the latest PR_REVIEW_CHECKLIST.md before pinging core maintainer on Slack for review. In order for the signoff PR check bot to trigger, you must follow the PR_REVIEW_CHECKLIST.md template correctly, including the phrase For PR verification, add the PR authors are responsible for ensuring that after merging, all GitHub Action jobs fully pass. A lot of the time, failures are just flakes and simply re-running the failed jobs will fix it. See GitHub's docs on re-running failed jobs 感谢你的贡献！请联系相应公司的 CODEOWNER 填写最新的 PR_REVIEW_CHECKLIST.md，然后再在 Slack 上联系核心维护者进行审阅。为了触发 signoff PR 检查机器人，你必须正确遵循 PR_REVIEW_CHECKLIST.md 模板，包括保留英文语句 如需进行 PR 验证，请为此 PR 添加 PR 作者有责任确保合并后所有 GitHub Action 任务完全通过。 很多时候失败只是偶发抖动（flake），重新运行失败的任务即可解决。参见 GitHub 关于重新运行失败任务的文档 |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
| - qwen3.5-fp4-b200-sglang-agentic-mtp | ||
| scenario-type: | ||
| - agentic-coding | ||
| description: |
There was a problem hiding this comment.
🔴 The new changelog entry sets pr-link to the literal string "PRLINK_PLACEHOLDER", which is not the documented placeholder and will persist as garbage if the PR link is never manually fixed post-merge (docs/configuration-procedures.md:254 permits only TBD pre-merge, to be replaced with the real PR URL immediately after creation). Fix: use pr-link: TBD (or the actual PR URL once known) instead of a non-standard placeholder string, per the documented convention.
Extended reasoning...
docs/configuration-procedures.md line 254 states the only permitted placeholder before a PR exists is pr-link: TBD, replaced with the real URL right after PR creation. This diff instead writes pr-link: PRLINK_PLACEHOLDER (perf-changelog.yaml line 6929), a string that does not match any tooling or convention referenced in the docs. Since perf-changelog.yaml is append-only and byte-sensitive (never edited retroactively per AGENTS.md conventions), if this placeholder is not swept and replaced before/at merge, it becomes a permanent malformed record in the historical changelog, breaking any downstream tooling or scripts that parse pr-link fields expecting either a real GitHub URL or the TBD sentinel.
Verification: normal. perf-changelog.yaml:6929 sets pr-link: PRLINK_PLACEHOLDER on the newly appended entry. The repo's changelog gate rejects this value: utils/validate_perf_changelog.py:144-160 (validate_added_pr_link, invoked from compare_entries at line 206) requires an appended entry's link to be either the exact https://github.com/SemiAnalysisAI/InferenceX/pull/<pr_number> or a member of… | normal.…
|
see unofficial run visualizer at https://inferencex.semianalysis.com/inference?unofficialRun=34168717312 |
1 similar comment
|
see unofficial run visualizer at https://inferencex.semianalysis.com/inference?unofficialRun=34168717312 |
1 participant
Summary
Update SGLang image for the Qwen3.5-397B-A17B NVFP4 B200 AgentX MTP recipe from
lmsysorg/sglang:v0.5.16-cu130(v0.5.16 release) tolmsysorg/sglang:nightly-dev-cu13-20260907-30705c00(2026-09-07 cu13 dev nightly).sha256:19b8fa1223cc339c1eae7a5b703f1a8c2543b5b119155bf3d7efaef18f77f007nightly-dev-cu13-<date>-<hash>convention, matching the current cu130 pin.Recipes touched:
qwen3.5-fp4-b200-sglang-agentic-mtpTest plan
🤖 Generated with Claude Code
Note
Low Risk
Benchmark config and changelog only; behavior change is limited to the pinned SGLang container for one recipe, with no auth or production runtime impact.
Overview
Updates the
qwen3.5-fp4-b200-sglang-agentic-mtpbenchmark recipe to uselmsysorg/sglang:nightly-dev-cu13-20260907-30705c00instead oflmsysorg/sglang:v0.5.16-cu130(cu13 dev nightly vs prior release pin). Engine flags, NEXTN MTP settings, golden acceptance length, and TP2/TP4 concurrency / HiCache search grids are left as-is.Adds a matching perf-changelog entry for the
agentic-codingscenario documenting the image digest, tag commit, and unchanged recipe parameters.The same config hunk also drops
dram-utilization: 0.80from theminimaxm3-fp8-h200-vllm-agentic-mtpagentic-coding sweep block immediately above the Qwen recipe (not called out in the PR description).Reviewed by Cursor Bugbot for commit 0a43cbe. Bugbot is set up for automated code reviews on this repo. Configure here.